Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

new aggregations UI #7102

Merged
merged 39 commits into from
Oct 29, 2024
Merged

new aggregations UI #7102

merged 39 commits into from
Oct 29, 2024

Conversation

JulianWielga
Copy link
Member

Describe your changes

Checklist before merge

  • Related issue ID is placed at the beginning of PR title in [brackets] (can be GH issue or Nu Jira issue)
  • Code is cleaned from temporary changes and commented out lines
  • Parts of the code that are not easy to understand are documented in the code
  • Changes are covered by automated tests
  • Showcase in dev-application.conf added to demonstrate the feature
  • Documentation added or updated
  • Added entry in Changelog.md describing the change from the perspective of a public distribution user
  • Added MigrationGuide.md entry in the appropriate subcategory if introducing a breaking change
  • Verify that PR will be squashed during merge

JulianWielga and others added 30 commits September 10, 2024 13:53
@JulianWielga
debug container update
545e05c
@JulianWielga
placeholders support
41167a7
@JulianWielga
optional fieldErrors
86cb5c6
@JulianWielga
cleanup
7aaa9a8
@JulianWielga
show errors in NodeRow
94599d8
@JulianWielga
extract findParamDefinitonByName
e2f1dad
@JulianWielga
replace aggregator & aggregateBy with row with editors
735e6ab
@JulianWielga
refactor & test fix
8675e38
@JulianWielga
fixed ace editor tabindex
217a745
@JulianWielga
groupBy
b2e04f2 
cleanup
@JulianWielga
fixed record parse
40ecbcb
@JulianWielga
cleanup
f6f8a78
@JulianWielga
fixed types error
adf97da
@JulianWielga
minor things
7e28c7a
@JulianWielga
Merge remote-tracking branch 'upstream/staging' into aggregate-ui
e3fcca1
@JulianWielga
json editor settings sync with aceWrapper
8231b25
@JulianWielga
test tweak
4b17cc8
@JulianWielga
Merge remote-tracking branch 'upstream/staging' into aggregate-ui
300234e
@JulianWielga
test tweak
1b9e796
@github-actions @JulianWielga
Updated snapshots (#6834)
4178e18 
Co-authored-by: JulianWielga <[email protected]>
@JulianWielga
minor types fixes
3f73fe8
@JulianWielga
fields labels
c1022f4
@JulianWielga
required fields
a64c0d3
@JulianWielga
presets fixed
ce7d43a
@JulianWielga
Merge remote-tracking branch 'upstream/staging' into aggregate-ui
7de29fd 
# Conflicts:
#	designer/client/cypress/e2e/__image_snapshots__/electron/Linux/Fragment should display dead-ended fragment correct #0.png
#	designer/client/src/common/ProcessUtils.ts
#	designer/client/src/http/HttpService.ts
@JulianWielga
sort type options
17c64fe
@JulianWielga
Merge branch 'staging' into preview/aggregate-ui
415c449
@JulianWielga
Merge branch 'staging' into preview/aggregate-ui
c110f56
@JulianWielga
Merge branch 'staging' into preview/aggregate-ui
14d3bff
@JulianWielga
Merge branch 'staging' into preview/aggregate-ui
b951ac5
@JulianWielga JulianWielga requested a review from Dzuming October 29, 2024 10:20
@github-actions github-actions bot added client client main fe ui labels Oct 29, 2024
@github-actions github-actions bot mentioned this pull request Oct 29, 2024
Merged
@github-actions GitHub Actions
Copy link
Contributor

created: #7105
⚠️ Be careful! Snapshot changes are not necessarily the cause of the error. Check the logs.

@github-actions @JulianWielga
Updated snapshots (#7105)
8d47f20 
Co-authored-by: JulianWielga <[email protected]>
@JulianWielga JulianWielga merged commit 84657ca into staging Oct 29, 2024
14 of 18 checks passed
@JulianWielga JulianWielga deleted the new-aggregate-ui branch October 29, 2024 13:20
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@Dzuming Dzuming Dzuming approved these changes

Assignees
No one assigned
Labels
client client main fe ui
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@JulianWielga @Dzuming